Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Note that Linux 5.3 or higher is required #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hamishmb
Copy link
Contributor

Note that Linux 5.3 or higher is required.

Also thus remove Debian 10 instructions, as Debian 10 uses an older kernel.

@trungnt2910
Copy link

I don't think this is relevant anymore, especially after darlinghq/darling#1207.

Current issues on WSL1 are Microsoft-specific, so theoretically Darling should run normally on a real Linux 4.4.0 kernel.

@hamishmb
Copy link
Contributor Author

Shall I test it and find out? Then, I can amend this pull request with the actual minimum kernel version.

@trungnt2910
Copy link

As of darlinghq/darling@0cada37 Darling is good enough to display a bash shell and information from neofetch on WSL1 (a subset of kernel version 4.4.0).

While it has not been tested on a real 4.4.0 kernel, it might be worth a try.

@hamishmb
Copy link
Contributor Author

hamishmb commented Sep 5, 2022

Might as well, I enjoy doing this kind of thing anyway. I'll update the pull request when I have some time - rather busy at the moment.

@hamishmb
Copy link
Contributor Author

Okay, can confirm that Debian 10 doesn't work, because the kernel headers are too old. In essence, I think this means that Darling (without this component perhaps) might run on a real 4.4.0 kernel, but might not build properly against the kernel headers for that 4.4.0 kernel.

While it might be possible to get Darling to compile if I edit the Cmake file, I think this is probably not worth doing, and probably wouldn't be a supported configuration anyway. I would attach a screenshot of the problem (see #off-topic in Discord), but don't seem to be able to.

At any rate, I can confirm that Debian 10 definitely doesn't work, so we should remove it. I can always add a note by the kernel version saying "(except for WSL1)" if that's what people want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants